Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ld/add local copro setup for e2e #71

Closed
wants to merge 15 commits into from

Conversation

leventdem
Copy link
Collaborator

The goal of this PR is to facilitate running e2e tests for a local running architecture.

@cla-bot cla-bot bot added the cla-signed label Dec 14, 2024
@leventdem leventdem changed the base branch from main to mano/cleanup December 14, 2024 17:22
@leventdem leventdem changed the base branch from mano/cleanup to main December 19, 2024 15:44
@leventdem
Copy link
Collaborator Author

For centralized mode the ERC20 test is passing in e2e

npm run test test/encryptedERC20/EncryptedERC20.ts                                                                                                                25s  ↑ ✱ ● ✭  ~/D/t/f/e2e 

> [email protected] test
> hardhat test test/encryptedERC20/EncryptedERC20.ts
  EncryptedERC20
    ✔ should mint the contract (9393ms)
Encrypting and proving in 22.3s
Verifying in 16.9s
    ✔ should transfer tokens between two users (59908ms)
Encrypting and proving in 22.3s
Verifying in 16.4s
    ✔ should not transfer tokens between two users (59844ms)
Encrypting and proving in 22.3s
Verifying in 16.9s
Encrypting and proving in 23s
Verifying in 16.6s
Encrypting and proving in 23s
Verifying in 17.7s
    ✔ should be able to transferFrom only if allowance is sufficient (169893ms)


  4 passing (5m)

@leventdem
Copy link
Collaborator Author

However for threshold mode the verification is still taking too much time, timeout has been increased to 60

> [email protected] test
> hardhat test test/encryptedERC20/EncryptedERC20.ts



  EncryptedERC20
    ✔ should mint the contract (12379ms)
Encrypting and proving in 22.5s
Verifying in 61.5s
    ✔ should transfer tokens between two users (128915ms)
Encrypting and proving in 24.1s
Verifying in 62.3s
    ✔ should not transfer tokens between two users (130930ms)
Encrypting and proving in 24.7s
Verifying in 59.7s
Encrypting and proving in 24.2s
Verifying in 61.1s
Encrypting and proving in 25.1s
Verifying in 62.3s
    ✔ should be able to transferFrom only if allowance is sufficient (351760ms)


  4 passing (11m)

@leventdem leventdem closed this Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants